-
Notifications
You must be signed in to change notification settings - Fork 24.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: place download section in toh to the top #36567
docs: place download section in toh to the top #36567
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this, @ajitsinghkaler! Looks good to me, but I'm also requesting review from another docs team member so we have consensus.
You can preview 3330a4f at https://pr36567-3330a4f.ngbuilds.io/. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like just the right thing to do.
I added the preview tag-- I'd like to see how it looks on the page.
@@ -13,6 +13,13 @@ When you’re done, users will be able to navigate the app like this: | |||
<img src='generated/images/guide/toh/nav-diagram.png' alt="View navigations"> | |||
</div> | |||
|
|||
<div class="alert is-helpful"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After looking at the preview, I think you might consider putting this link above the image, to make it more visible at the top of the page.
I would put it right after the bullet list.
3330a4f
to
90537eb
Compare
@jbogarthyde @kapunahelewong added the suggestions, please check and add the merge label |
You can preview 90537eb at https://pr36567-90537eb.ngbuilds.io/. |
LGTM |
@StephenFluin can you please review this |
@StephenFluin Can you please review this |
@jbogarthyde @kapunahelewong can we get this merged |
Thanks for the tag, @ajitsinghkaler. I've requested @aikidave to review to help move this forward. |
@aikidave @StephenFluin can you help me get this merged |
this is part of a larger effort to standardise download sections on angular.io This commit partially addresses angular#35459
90537eb
to
c7bf2fb
Compare
You can preview c7bf2fb at https://pr36567-c7bf2fb.ngbuilds.io/. |
this is part of a larger effort to standardise download sections on angular.io This commit partially addresses angular#35459 PR Close angular#36567
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
this is part of a larger effort to standardise download sections on angular.io This commit partially addresses angular#35459 PR Close angular#36567
this is part of a larger effort to standardise download sections on angular.io
This commit partially addresses #35459
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
No standardized download exampes
Issue Number: #35459
What is the new behavior?
Standardized download examples
Does this PR introduce a breaking change?
Other information